Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix(types): should unwrap tuple correctly #3820

Merged
merged 3 commits into from
Oct 26, 2022
Merged

fix(types): should unwrap tuple correctly #3820

merged 3 commits into from
Oct 26, 2022

Conversation

hyf0
Copy link
Contributor

@hyf0 hyf0 commented May 23, 2021

fix #3819.

@HcySunYang HcySunYang changed the title fix(reactivity): UnwrapRefSimple<T> should not unwrap tuple fix(types): should unwrap tuple correctly May 23, 2021
@HcySunYang HcySunYang requested a review from pikax May 23, 2021 05:07
@hyf0 hyf0 force-pushed the fix_should_not_unwrap_tuple branch 3 times, most recently from 7b58d85 to a7b5da2 Compare May 23, 2021 14:47
@hyf0 hyf0 mentioned this pull request May 27, 2021
Closed
@hyf0 hyf0 force-pushed the fix_should_not_unwrap_tuple branch from a7b5da2 to ce0aabe Compare May 27, 2021 10:15
@antfu antfu added the ready to merge The PR is ready to be merged. label Aug 11, 2021
@hyf0 hyf0 force-pushed the fix_should_not_unwrap_tuple branch 4 times, most recently from 09e24ca to b041a8a Compare September 12, 2021 09:36
@hyf0 hyf0 force-pushed the fix_should_not_unwrap_tuple branch from b041a8a to c77a050 Compare October 22, 2021 03:18
@hyf0 hyf0 force-pushed the fix_should_not_unwrap_tuple branch from c77a050 to 9909359 Compare October 22, 2021 03:22
@JensDll JensDll mentioned this pull request Nov 3, 2021
Closed
1 task
@yyx990803 yyx990803 merged commit e816812 into vuejs:main Oct 26, 2022
chrislone pushed a commit to chrislone/core that referenced this pull request Feb 4, 2023
@hyf0
fix(types): should unwrap tuple correctly (vuejs#3820)
7a23d07 
fix vuejs#3819
zhangzhonghe pushed a commit to zhangzhonghe/core that referenced this pull request Apr 12, 2023
@hyf0 @zhangzhonghe
fix(types): should unwrap tuple correctly (vuejs#3820)
ad05213 
fix vuejs#3819
@skirtles-code skirtles-code mentioned this pull request Apr 11, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cexbrayat cexbrayat cexbrayat left review comments

@HcySunYang HcySunYang HcySunYang approved these changes

@antfu antfu antfu approved these changes

@pikax pikax pikax approved these changes

Assignees
No one assigned
Labels
ready to merge The PR is ready to be merged. scope: types
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

unexpected unwrapping for tuple
6 participants
@hyf0 @cexbrayat @pikax @antfu @HcySunYang @yyx990803